-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Generalize reverse symmetry concept #408
base: develop
Are you sure you want to change the base?
Conversation
shared/src/main/java/com/faforever/neroxis/mask/BooleanMask.java
Outdated
Show resolved
Hide resolved
f082a8a
to
338051a
Compare
f026eaf
to
7d98fcb
Compare
Alright @clivepaterson I think that I was able to fix the underlying issues with diag and the path function. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as off-topic.
This comment was marked as off-topic.
Ha yeah there also was a little bit of scope creep with some changes to the visual debugger to make it easier to verify some of the changes. |
Also sorry on the broker build had messed up the rebase and didn't notice |
Interesting it looks like an angle for maybe the terrain symmetry is off. Can you add the map names here as copyable text? |
Yep I have updated the comment with the meta data and name. This one is also a small one, also looks good for testing: I'm guessing it's prolly a small fix somewhere in the SymmetryUtil functions maybe, or one layer having issues, not sure. |
No description provided.